Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SteelSeries Arctis 7 Battery Support + 2019 Model Support #40

Merged
merged 14 commits into from Aug 8, 2019

Conversation

jamesj999
Copy link
Contributor

@jamesj999 jamesj999 commented Aug 4, 2019

Added battery support and support for 2019 variant of SteelSeries Arctis 7.

@jamesj999 jamesj999 changed the title Feature/steelseries arctis 7 2019 SteelSeries Arctis 7 Battery Support + 2019 Model Support Aug 4, 2019
@jamesj999
Copy link
Contributor Author

Fixes #35

@Sapd
Copy link
Owner

Sapd commented Aug 5, 2019

Thank you!
Is the Request still Work-In-Progress? (As some commits were added some hours ago).
Also the README needs an update, I can also take care of it, though.

@jamesj999
Copy link
Contributor Author

Hello there!

Apologies, we were slightly premature with the request - we spotted a couple of issues and fixed them. We're just double checking the fully charged value as I've seen it go over 100% so we're verifying on a second headset and will cap it at 100% if necessary. I'll update the readme now and will let you know when we've checked the battery percentage issue.

Thanks,
James

@raiscan
Copy link

raiscan commented Aug 5, 2019

Thank you!

Thank you! We had a good basis to get our headset battery in Linux thanks to this project.

Fix 50-steelseries-arctis-7.rules UDev rule, According to Udev versions
used in Ubuntu 19.04. Previous version did not work.
@raiscan
Copy link

raiscan commented Aug 6, 2019

After experimenting, I believe their battery percentage response isn't calibrated very well or calibrated to a reference battery; mine never goes above 99, while @jamesj999 's can reach all the way to 116 before capping out. The SteelSeries UI in Windows uses a simple battery segment indicator so I imagine they're aware it's not particularly accurate.

When I get home tonight I'll just limit the max output to 100. It'll be good enough for anyone's purposes; especially since unplugging James's heatset immediately drops it lower than 100 :)

@Sapd
Copy link
Owner

Sapd commented Aug 6, 2019

Yes I think this is not uncommon, there are two ways to measure how full a battery is.

The first (halfway accurate) method is to empty the battery completely and charge it until the maximum voltage is reached and count the watt-hours. (Now you can simply integrate the watt-hours while using the headset and know how much of the maximum number is left).

The second method (probably with most headsets) is to simply measure the voltage. It's very inaccurate and not really linear. Therefore, it may well be that the value is also inaccurate.

Fix 50-steelseries-arctis-7-2019.rules (in line with the changes made for non-2019)
@raiscan
Copy link

raiscan commented Aug 7, 2019

Hello! I forgot last night. I've made some changes to the arctis 7 rules due to the old udev rules not working (See commits 669aac0 & 2128b78). I have also capped the battery readout on 100% to both arctis 7 and arctis 7 2019 as they seem to be pretty darn similar and we've seen it on one at least.

Please let me know if there's anything you'd like us to change, but otherwise I think we're done :) Thanks!

@Sapd Sapd merged commit 1dbc9ef into Sapd:master Aug 8, 2019
@Sapd
Copy link
Owner

Sapd commented Aug 8, 2019

Looks good, maybe the UDEV rules need to be adjusted for other devices, but this pull request is fine.

@raiscan raiscan deleted the feature/steelseries-arctis-7-2019 branch August 9, 2019 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants